Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

use rustc-fake as RUSTC_WRAPPER #1576

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

csmoe
Copy link
Member

@csmoe csmoe commented Apr 23, 2023

  1. the 1-feature.patch makes cargo compile-able on OS without openssl pre-installed like osx;
  2. RUSTC=rustc_fake makes the build script which depends on Command::new("RUSTC") failed like autocfg

@csmoe csmoe force-pushed the rustc_wrapper branch 5 times, most recently from 3f250a5 to 4528695 Compare April 26, 2023 13:11
@csmoe csmoe changed the title replace rustc_fake with rustc_wrapper restore RUSTC after rustc-fake substitution Apr 26, 2023
@csmoe csmoe changed the title restore RUSTC after rustc-fake substitution restore RUSTC/RUSTDOC after rustc-fake substitution Apr 26, 2023
@csmoe
Copy link
Member Author

csmoe commented Apr 26, 2023

r? @Kobzol

@csmoe csmoe force-pushed the rustc_wrapper branch 5 times, most recently from 933e118 to b73177e Compare April 26, 2023 16:15
@csmoe csmoe changed the title restore RUSTC/RUSTDOC after rustc-fake substitution use rustc-fake as RUSTC_WRAPPER Apr 26, 2023
@Kobzol
Copy link
Contributor

Kobzol commented Apr 29, 2023

Hi, could you maybe explain a little bit what is the motivation for this change (and what does it even do?)?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants